Rollup of 16 pull requests#157121
Conversation
…d named lifetimes
… so just encode the fields we need
This makes the proc-macro RPC protocol platform independent as necessary for running proc macros and rustc on different architectures.
This shrinks the proc-macro ABI a bit. Nothing on the client side needs to allocate handles. This will share the counters between proc-macros, which may cause us to run out of handles earlier, but 4 bilion handles per process is still a lot. Rust-analyzer might run out at some point, but it can recover by restarting the proc-macro-server, and it could already run out before this change anyway,
`CLFLUSHOPT` is a standalone x86 CPUID feature (Intel Skylake+ / AMD Excavator+, ~2015) not part of any psABI microarchitecture level, so it needs its own unstable target feature, gated behind `clflushopt_target_feature`. This is the compiler half of exposing the `_mm_clflushopt` intrinsic; the stdarch half is blocked on this landing. Also bless two UI test references (check-cfg/target_feature and target-feature/invalid-attribute) that enumerate the full set of valid target features, since `clflushopt` is now part of that list.
Wire up runtime detection for the `clflushopt` feature: add it to the `is_x86_feature_detected!` feature list (gated on `clflushopt_target_feature`), enable it from CPUID leaf 7 EBX bit 23, and add it to the std_detect x86-specific dump test.
|
💔 Test for c24e9c9 failed: CI. Failed job:
|
|
A job failed! Check out the build log: (web) (plain enhanced) (plain) Click to see the possible cause of the failure (guessed by this bot) |
|
@bors retry |
This comment has been minimized.
This comment has been minimized.
What is this?This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.Comparing 6368fd5 (parent) -> 6eda741 (this PR) Test differencesShow 257 test diffsStage 1
Stage 2
Additionally, 244 doctest diffs were found. These are ignored, as they are noisy. Job group index
Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard 6eda7419e71fdbc1185ed5be7e6bff1a474ab5cd --output-dir test-dashboardAnd then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
|
📌 Perf builds for each rolled up PR:
previous master: 6368fd52cb In the case of a perf regression, run the following command for each PR you suspect might be the cause: |
|
Finished benchmarking commit (6eda741): comparison URL. Overall result: ❌✅ regressions and improvements - BENCHMARK(S) FAILEDOur benchmarks found a performance regression caused by this PR. Next Steps:
@rustbot label: +perf-regression ❗ ❗ ❗ ❗ ❗
❗ ❗ ❗ ❗ ❗ cc @rust-lang/wg-compiler-performance Instruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
Max RSS (memory usage)Results (primary -1.3%, secondary -0.2%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 2.2%, secondary 7.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (secondary -0.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 511.654s -> 518.609s (1.36%) |
|
The failed benchmark is caused by #149195 (comment) |
View all comments
Successful merges:
ambiguous_glob_importslint into a hard error #149195 (resolve: Partially convertambiguous_glob_importslint into a hard error)DefPathTable, useLocalDefIdinstead ofDefIndex#156963 (definitions: removeDefPathTable, useLocalDefIdinstead ofDefIndex)Path::is_empty#157065 (StabilizePath::is_empty)clflushoptx86 target feature #157098 (Add theclflushoptx86 target feature)with_cx()toCompilerInterface#157116 (rustc_public: addwith_cx()toCompilerInterface)resolve_pin_drop_sugar_impl_item#157119 (ast_lowering: Simplifyresolve_pin_drop_sugar_impl_item)Failed merges:
r? @ghost
Create a similar rollup